-
Notifications
You must be signed in to change notification settings - Fork 126
Add integration tests to each C++ library. #77
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
I don't know why all the stuff from the merge ended up in this PR or how to get rid of it... |
1f1dcd0 to
d68b186
Compare
|
I had to squash this into a single commit to fix the issue with the merge commit being included. Sorry about that! |
d68b186 to
bdcd3f3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mainly Copyright police. Some of my copyright comments have ?s -- while the AndroidManifest.xml should have copyrights (if we're following the other google open source projects) I wasn't sure if the strings.xml and main.xml should. I would think so...
|
It looks like the "(api)/integration_test/src" directories are missing all the *_framework files. Internally they're symlinks, but it looks like they didn't get included in this PR in any form. Edit: Looks like other symlinks are missing as well, such as download_googletest.py. |
download_googletest.py and the symlinks in src are populated by the setup_integration_tests.py script, which is run by the build process in each mode of building (Podfile, build.gradle, and cmake file) before downloading googletest. |
No harm in adding copyright to as many files as possible - I added it to all xml, bat, gradlew, and build.gradle files. |
|
Added copyrights and fixed the Info.plist files, PTAL. |
Ah, I see why it's not running for me - the testapp builder copies the integration test project to another directory to work clean. The location of the setup script is determined relative to the cmake file in the test project, and only tries to run if found. Could be fixed by looking relative to the SDK/repo directory instead, but I'm not sure how easy it is to get a reference to that in each build file (e.g. from the pod file). It's not difficult to work around if this is intended behaviour, though - I could change the testapp builder to copy the whole repo instead of just the integration test project, as it's not very large (~50mb). |
It's intended - I think the best way to do it might be to run the setup script, and then copy the integration tests to a working directory. |
|
(Also that would only be necessary when running the integration tests against a binary SDK. You can run the integration tests against the source SDK directly inside the repo.) |
|
@a-maurice @vimanyu @DellaBitta ping? |
Added integration test build for Desktop and Android. iOS requires adding dependencies manually right now and so does not yet build automatically.
Tested on Mac and Linux for both desktop and Android. Not yet tested on Windows, but will do that in a follow-up after we hook it up to the GHA.
Note that there are a few manual things that have to happen before the integration test can be run: